Skip to content

Conversation

Nadrieril
Copy link
Member

This reverts the hack in #133889 now that Pin's field is no longer public.

Fixes #143468.

r? @compiler-errors

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Jul 20, 2025
@rustbot

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

This reverts the hack in rust-lang#133889
now that `Pin`'s field is no longer public.
Copy link
Member

@compiler-errors compiler-errors left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me when CI is green

@Nadrieril
Copy link
Member Author

@bors r=compiler-errors rollup

@bors
Copy link
Collaborator

bors commented Jul 20, 2025

📌 Commit 3567ab1 has been approved by compiler-errors

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 20, 2025
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jul 20, 2025
…ler-errors

Don't consider unstable fields always-inhabited

This reverts the hack in rust-lang#133889 now that `Pin`'s field is no longer public.

Fixes rust-lang#143468.

r? `@compiler-errors`
bors added a commit that referenced this pull request Jul 20, 2025
Rollup of 7 pull requests

Successful merges:

 - #142097 (gpu offload host code generation)
 - #143906 (Miri: non-deterministic floating point operations in `foreign_items`)
 - #144144 (tests: Skip supported-crate-types test on musl hosts)
 - #144159 (opt-dist: change build_dir field to be an actual build dir)
 - #144162 (Debug impls for DropElaborators)
 - #144189 (Add non-regression test for #144168)
 - #144216 (Don't consider unstable fields always-inhabited)

r? `@ghost`
`@rustbot` modify labels: rollup
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jul 20, 2025
…ler-errors

Don't consider unstable fields always-inhabited

This reverts the hack in rust-lang#133889 now that `Pin`'s field is no longer public.

Fixes rust-lang#143468.

r? ``@compiler-errors``
bors added a commit that referenced this pull request Jul 21, 2025
Rollup of 6 pull requests

Successful merges:

 - #142097 (gpu offload host code generation)
 - #143906 (Miri: non-deterministic floating point operations in `foreign_items`)
 - #144144 (tests: Skip supported-crate-types test on musl hosts)
 - #144162 (Debug impls for DropElaborators)
 - #144189 (Add non-regression test for #144168)
 - #144216 (Don't consider unstable fields always-inhabited)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Jul 21, 2025
Rollup of 8 pull requests

Successful merges:

 - #144144 (tests: Skip supported-crate-types test on musl hosts)
 - #144159 (opt-dist: change build_dir field to be an actual build dir)
 - #144162 (Debug impls for DropElaborators)
 - #144189 (Add non-regression test for #144168)
 - #144216 (Don't consider unstable fields always-inhabited)
 - #144229 (Miri subtree update)
 - #144230 (Option::as_slice: fix comment)
 - #144235 (Fix run-make tests on musl hosts)

r? `@ghost`
`@rustbot` modify labels: rollup
bors added a commit that referenced this pull request Jul 21, 2025
Rollup of 8 pull requests

Successful merges:

 - #144144 (tests: Skip supported-crate-types test on musl hosts)
 - #144159 (opt-dist: change build_dir field to be an actual build dir)
 - #144162 (Debug impls for DropElaborators)
 - #144189 (Add non-regression test for #144168)
 - #144216 (Don't consider unstable fields always-inhabited)
 - #144229 (Miri subtree update)
 - #144230 (Option::as_slice: fix comment)
 - #144235 (Fix run-make tests on musl hosts)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 830b823 into rust-lang:master Jul 21, 2025
11 checks passed
@rustbot rustbot added this to the 1.90.0 milestone Jul 21, 2025
rust-timer added a commit that referenced this pull request Jul 21, 2025
Rollup merge of #144216 - Nadrieril:revert-pin-hack, r=compiler-errors

Don't consider unstable fields always-inhabited

This reverts the hack in #133889 now that `Pin`'s field is no longer public.

Fixes #143468.

r? ```@compiler-errors```
Muscraft pushed a commit to Muscraft/rust that referenced this pull request Jul 21, 2025
…ler-errors

Don't consider unstable fields always-inhabited

This reverts the hack in rust-lang#133889 now that `Pin`'s field is no longer public.

Fixes rust-lang#143468.

r? ```@compiler-errors```
Muscraft pushed a commit to Muscraft/rust that referenced this pull request Jul 21, 2025
Rollup of 8 pull requests

Successful merges:

 - rust-lang#144144 (tests: Skip supported-crate-types test on musl hosts)
 - rust-lang#144159 (opt-dist: change build_dir field to be an actual build dir)
 - rust-lang#144162 (Debug impls for DropElaborators)
 - rust-lang#144189 (Add non-regression test for rust-lang#144168)
 - rust-lang#144216 (Don't consider unstable fields always-inhabited)
 - rust-lang#144229 (Miri subtree update)
 - rust-lang#144230 (Option::as_slice: fix comment)
 - rust-lang#144235 (Fix run-make tests on musl hosts)

r? `@ghost`
`@rustbot` modify labels: rollup
github-actions bot pushed a commit to rust-lang/miri that referenced this pull request Jul 22, 2025
Rollup of 8 pull requests

Successful merges:

 - rust-lang/rust#144144 (tests: Skip supported-crate-types test on musl hosts)
 - rust-lang/rust#144159 (opt-dist: change build_dir field to be an actual build dir)
 - rust-lang/rust#144162 (Debug impls for DropElaborators)
 - rust-lang/rust#144189 (Add non-regression test for rust-lang/rust#144168)
 - rust-lang/rust#144216 (Don't consider unstable fields always-inhabited)
 - rust-lang/rust#144229 (Miri subtree update)
 - rust-lang/rust#144230 (Option::as_slice: fix comment)
 - rust-lang/rust#144235 (Fix run-make tests on musl hosts)

r? `@ghost`
`@rustbot` modify labels: rollup
@Nadrieril Nadrieril deleted the revert-pin-hack branch July 23, 2025 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exhaustive matching doesn't work correctly for Coroutines with never_type variants
6 participants